Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: added documentation for premium notifications #28

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

vickywane
Copy link

@vickywane vickywane commented May 25, 2021

This pull request adds the documentation of the new premium notifications within the Ambianic PWA UI here into the Ambianic Docs

The new changes are introduced in the premium-services.md file which is linked to the Quickstart Guide in the Ambianic Premium Services section.

@gitpod-io
Copy link

gitpod-io bot commented May 25, 2021

@commit-lint
Copy link

commit-lint bot commented May 25, 2021

Features

  • added documentation for premium notifications (5beac44)

Update

  • expanded docs and added relevant images (9defb90)
  • linked premium-services to quickstart (196c7a9)

Bug Fixes

Contributors

vickywane

Commit-Lint commands

You can trigger Commit-Lint actions by commenting on this PR:

  • @Commit-Lint merge patch will merge dependabot PR on "patch" versions (X.X.Y - Y change)
  • @Commit-Lint merge minor will merge dependabot PR on "minor" versions (X.Y.Y - Y change)
  • @Commit-Lint merge major will merge dependabot PR on "major" versions (Y.Y.Y - Y change)
  • @Commit-Lint merge disable will desactivate merge dependabot PR
  • @Commit-Lint review will approve dependabot PR
  • @Commit-Lint stop review will stop approve dependabot PR

@vickywane vickywane requested a review from ivelin May 25, 2021 12:50
@vickywane vickywane self-assigned this May 25, 2021
@vickywane vickywane added the documentation Improvements or additions to documentation label May 25, 2021
Copy link
Contributor

@ivelin ivelin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vickywane make this PR available for preview on your main fork with all CI checks passing.

Added a few inline comments for language styling.

docs-md/users/premium-services.md Outdated Show resolved Hide resolved
docs-md/users/premium-services.md Outdated Show resolved Hide resolved
docs-md/users/premium-services.md Outdated Show resolved Hide resolved
docs-md/users/premium-services.md Outdated Show resolved Hide resolved
docs-md/users/premium-services.md Outdated Show resolved Hide resolved
* feat: added documentation for premium notifications

* update: expanded docs and added relevant images

* update: linked premium-services to quickstart
@vickywane
Copy link
Author

@ivelin This is my fork of this pull request available for review.

All CI checks passed in the pull request here before it was merged.

@ivelin
Copy link
Contributor

ivelin commented May 27, 2021 via email

Copy link
Contributor

@ivelin ivelin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vickywane The text looks good. See comments on organizing sections and menus.

mkdocs.yml Show resolved Hide resolved
Copy link
Contributor

@ivelin ivelin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its looking good. Only one comment left, which I thought I made in the previous review, but I don't see it in the history. Other than that, we can merge this docs PR as soon as the code is ready.

docs-md/users/quickstart.md Show resolved Hide resolved
Copy link
Contributor

@ivelin ivelin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's looking good. Will merge together with the cloud api code when ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants